-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New color palette #929
base: main
Are you sure you want to change the base?
New color palette #929
Conversation
✅ Deploy Preview for acre-dapp-testnet ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for acre-dapp ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
DApp has been redesigned and projects use a new color palette. Let's update the color palette in the code. However, let's leave the old palette until we update the base components. This is the simplest solution to avoid complications with new colors.
5fb7d6f
to
2673faa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code looks good to me. I'll go through the dapp soon to make sure everthing looks good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me 🚀
I looked through the dApp and didn't spot any issues. Leaving approval for @r-czajkowski
Ref AENG-9
DApp has been redesigned and projects use a new color palette. Let's update the color palette in the code. However, let's leave the old palette until we update the base components. This is the simplest solution to avoid complications with new colors.
How the palette was updated:
brand.400
->acre.50
grey.700
->text.primary
orbrown.100
grey.500
->text.tertiary
green.500
->green.50
red.500
->red.50
gold.100
->surface.2
gold.200
->surface.3
gold.300
->surface.4
gold.500
->surface.5
Note, that the design has changed, so some components have been matched to the new designs.
Testing
Please go through the dApp to make sure nothing looks wrong. Components such as links, button alert, modal, card will be updated soon.